-
Notifications
You must be signed in to change notification settings - Fork 51
Upgrade ember-stargate
#3268
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade ember-stargate
#3268
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| "@embroider/addon-dev": "^7.1.5", | ||
| "@eslint/js": "^9.23.0", | ||
| "@glimmer/component": "^1.1.2", | ||
| "@glimmer/component": "^2.0.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you might need to add global override for this, similar to what we have in cloud-ui https://github.com/hashicorp/cloud-ui/blob/main/package.json#L55
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're upgrading ember-a11y-refocus in another branch. Would waiting on that to merge solve the issue?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it might yeah as I suggested this solution to @shleewhite when we paired on that upgrade
ember-stargate & @glimmer/component
3851d8f to
5d044c7
Compare
beddeda to
219ec81
Compare
219ec81 to
6615eaf
Compare
5fc6964 to
5ef8b4f
Compare
6615eaf to
abab065
Compare
ember-stargate & @glimmer/componentember-stargate
| import { BORDER_WIDTH } from './index.ts'; | ||
| import type Owner from '@ember/owner'; | ||
|
|
||
| import type Owner from '@ember/owner'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change seems unrelated. Perhaps we should drop it?
abab065 to
22ad9c2
Compare
π Summary
If merged, this PR will upgrade dependencies ahead of the 5.0.0 release
π οΈ Detailed description
ember-stargateto1.0.2π External links
Jira ticket: HDS-4936
π Component checklist
π¬ Please consider using conventional comments when reviewing this PR.
π PCI review checklist
Examples of changes to controls include access controls, encryption, logging, etc.
Examples include changes to operating systems, ports, protocols, services, cryptography-related components, PII processing code, etc.